Skip to content

tox: drop non-working Python 3.6 from tox tests#1661

Merged
FrostyX merged 1 commit intorpm-software-management:mainfrom
praiskup:praiskup-fix-tox
Dec 1, 2025
Merged

tox: drop non-working Python 3.6 from tox tests#1661
FrostyX merged 1 commit intorpm-software-management:mainfrom
praiskup:praiskup-fix-tox

Conversation

@praiskup
Copy link
Member

@praiskup praiskup commented Dec 1, 2025

We no longer support Mock on RHEL 7 (main branch) anyway. RHEL 8 runs on Python 3.9.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request correctly removes Python 3.6 from the tox test environments, and the change in tox.ini is accurate. As a follow-up, I recommend simplifying mock/requirements.txt. Since Python < 3.9 is no longer tested, the conditional dependency on pyroute2 can be made unconditional, which would improve maintainability by replacing pyroute2==0.5.3; python_version < "3.9" and pyroute2; python_version >= "3.9" with just pyroute2.

We no longer support Mock on RHEL 7 (`main` branch) anyway.
RHEL 8 runs on Python 3.9.
@FrostyX FrostyX merged commit a06db88 into rpm-software-management:main Dec 1, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants